Skip to content

Conversation

@fs-eire
Copy link
Contributor

@fs-eire fs-eire commented Jul 21, 2025

Description

Motivation and Context

Fix the build break on Windows+Ninja

@fs-eire fs-eire changed the title [WIP] upgrade emsdk to v4.0.11 upgrade emsdk to v4.0.11 Jul 21, 2025
@fs-eire fs-eire requested a review from Copilot July 21, 2025 22:20
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR upgrades the Emscripten SDK (emsdk) from version 4.0.8 to 4.0.11 to fix a build break on Windows+Ninja. The upgrade involves updating version references across multiple build configuration files and simplifying WebAssembly async wrapper code to align with newer emsdk capabilities.

Key changes:

  • Updated emsdk version references from 4.0.8 to 4.0.11 across build scripts and configuration files
  • Simplified WebAssembly async function wrapping by removing complex debug/release build handling logic
  • Updated compiler flag syntax and added character conversion warning handling for compatibility

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tools/ci_build/github/azure-pipelines/templates/linux-wasm-ci.yml Updated emsdk install/activate commands to use version 4.0.11
tools/ci_build/build_args.py Changed default emsdk_version argument from 4.0.8 to 4.0.11
onnxruntime/wasm/pre-async.js Simplified async wrapper implementation by removing debug/release build handling
cmake/onnxruntime_webassembly.cmake Updated exception catching flag syntax and configuration
cmake/onnxruntime_unittests.cmake Added character conversion warning suppression
cmake/external/emsdk Updated submodule commit to emsdk 4.0.11
cmake/CMakeLists.txt Added character conversion compiler flag check
.gitmodules Updated emsdk submodule branch from 4.0.8 to 4.0.11

@fs-eire fs-eire merged commit b214da5 into main Jul 26, 2025
95 of 97 checks passed
@fs-eire fs-eire deleted the fs-eire/upgrade-emsdk-4.0.11 branch July 26, 2025 21:25
sanketkaleoss pushed a commit to sanketkaleoss/onnxruntime that referenced this pull request Aug 11, 2025
### Description

### Motivation and Context

Fix the build break on Windows+Ninja
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants